Skip to content

diagnostics_channel: add diagnostics channels for web locks#62123

Open
IlyasShabi wants to merge 2 commits intonodejs:mainfrom
IlyasShabi:ishabi/web-lock-dc
Open

diagnostics_channel: add diagnostics channels for web locks#62123
IlyasShabi wants to merge 2 commits intonodejs:mainfrom
IlyasShabi:ishabi/web-lock-dc

Conversation

@IlyasShabi
Copy link
Member

This PR adds diagnostics channels support for locks.request() giving visibility into the lock request lifecycle.

Four channels are emitted:

  • locks.request.start when a request begins
  • locks.request.grant when the lock is granted
  • locks.request.miss when the lock is not immediately available
  • locks.request.end when the request finishes

@nodejs-github-bot nodejs-github-bot added the needs-ci PRs that need a full CI run. label Mar 5, 2026
@IlyasShabi IlyasShabi marked this pull request as draft March 5, 2026 22:17
@IlyasShabi IlyasShabi force-pushed the ishabi/web-lock-dc branch from c92cc75 to 8ce0c9b Compare March 5, 2026 22:26
@IlyasShabi IlyasShabi marked this pull request as ready for review March 5, 2026 22:28
@IlyasShabi IlyasShabi changed the title locks: add diagnostics channels for web locks diagnostics_channel: add diagnostics channels for web locks Mar 5, 2026
@codecov
Copy link

codecov bot commented Mar 6, 2026

Codecov Report

❌ Patch coverage is 89.47368% with 6 lines in your changes missing coverage. Please review.
✅ Project coverage is 89.65%. Comparing base (da5efc4) to head (ca7fc4d).
⚠️ Report is 121 commits behind head on main.

Files with missing lines Patch % Lines
lib/internal/locks.js 89.47% 6 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main   #62123      +/-   ##
==========================================
+ Coverage   88.84%   89.65%   +0.81%     
==========================================
  Files         674      676       +2     
  Lines      204957   206604    +1647     
  Branches    39309    39554     +245     
==========================================
+ Hits       182087   185232    +3145     
+ Misses      15088    13502    -1586     
- Partials     7782     7870      +88     
Files with missing lines Coverage Δ
lib/internal/locks.js 94.39% <89.47%> (-0.70%) ⬇️

... and 250 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@IlyasShabi IlyasShabi requested review from BridgeAR and Qard March 6, 2026 13:01
@legendecas legendecas added the diagnostics_channel Issues and PRs related to diagnostics channel label Mar 6, 2026
@IlyasShabi IlyasShabi requested a review from legendecas March 10, 2026 14:12
Copy link
Member

@legendecas legendecas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry I forgot to hit the submit button. LGTM with a minor suggestion.

@IlyasShabi IlyasShabi requested a review from legendecas March 10, 2026 14:27
@IlyasShabi IlyasShabi added the request-ci Add this label to start a Jenkins CI on a PR. label Mar 10, 2026
@github-actions github-actions bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Mar 10, 2026
@nodejs-github-bot
Copy link
Collaborator

@nodejs-github-bot
Copy link
Collaborator

@nodejs-github-bot
Copy link
Collaborator

@IlyasShabi IlyasShabi added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Mar 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author ready PRs that have at least one approval, no pending requests for changes, and a CI started. diagnostics_channel Issues and PRs related to diagnostics channel needs-ci PRs that need a full CI run.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants